Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: cherry-pick 7e5c7b5964 from v8 #25646

Merged
merged 1 commit into from Oct 9, 2020
Merged

Conversation

miniak
Copy link
Contributor

@miniak miniak commented Sep 26, 2020

[compiler] Fix bug in SimplifiedLowering's overflow computation

It's unsound to ignore -0 inputs:
-0 - INT32_MIN is outside of INT32 range.

Bug: chromium:1126249
Change-Id: I3b92f16c1201705780acb0359975329aa2ca34d1
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2404452
Reviewed-by: Tobias Tebbi tebbi@chromium.org
Commit-Queue: Georg Neis neis@chromium.org
Cr-Commit-Position: refs/heads/master@{#69877}

Notes: Security: backported fix for 1126249.

@miniak miniak added the 8-x-y label Sep 26, 2020
@miniak miniak requested a review from a team September 26, 2020 21:08
@miniak miniak requested a review from a team as a code owner September 26, 2020 21:08
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Sep 26, 2020
@miniak miniak force-pushed the cherry-pick/8-x-y/v8/f1031d8fd0 branch from bb15ac6 to f09d78b Compare September 27, 2020 01:00
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Sep 27, 2020
@jkleinsc jkleinsc added the backport-check-skip Skip trop's backport validity checking label Sep 28, 2020
Copy link
Member

@zcbenz zcbenz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI requires this patch to be updated:

======================================================================
There were changes to the patches when applying.
Check the CI artifacts for a patch you can apply to fix it.
======================================================================

@miniak miniak force-pushed the cherry-pick/8-x-y/v8/f1031d8fd0 branch from f09d78b to 7db5b4f Compare October 7, 2020 00:07
@miniak miniak force-pushed the cherry-pick/8-x-y/v8/f1031d8fd0 branch from 7db5b4f to 227b14a Compare October 7, 2020 00:21
@miniak miniak requested a review from zcbenz October 7, 2020 00:22
@zcbenz zcbenz merged commit d47f7c3 into 8-x-y Oct 9, 2020
@release-clerk
Copy link

release-clerk bot commented Oct 9, 2020

Release Notes Persisted

Security: backported fix for 1126249.

@zcbenz zcbenz deleted the cherry-pick/8-x-y/v8/f1031d8fd0 branch October 9, 2020 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8-x-y backport-check-skip Skip trop's backport validity checking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants